script.js ➔ acceptCookies - Code Metrics - Inspection of "feat(GoogleAnalytics Feature, CookieNotification):..." - flyntwp/flynt-starter-theme - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — BaseComponent-BlockCookieNotif... ( e5b0b7...17adf5 )

unknown

created 2017-06-23 14:58 UTC

script.js ➔ acceptCookies A

↳ Parent: Components/BlockCookieNotification/Partials/AcceptButton/script.js

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
nc	1
nop	3
dl	0
loc	9
rs	9.6666
c	1
b	0
f	0

1 Function

Rating	Name	Duplication	Size	Complexity
A	script.js ➔ ... ➔	0	7	1

import $ from 'jquery'

export function getOptions (optionContainer) {

    return {
      expireDays: 7,
      cookieName: 'cookies_accepted'
    }
  }

export function acceptCookies ($container, cookieName, expireDays) {
  return function (e) {

    const date = new Date()
    date.setTime(date.getTime() + (expireDays*24*60*60*1000))
    document.cookie = cookieName + "=true; expires=" + date.toGMTString()

    $container.remove()
  }
}

export function checkCookies ($container, cookieName) {
    const cookiesAccepted = getCookieValue(cookieName)
    
    if (!cookiesAccepted) {
      $container.addClass('cookieNotification-isVisible')
    } else {
      $container.remove()
    }
  }



function getCookieValue (cookie) {
  const value = "; " + document.cookie
  const parts = value.split('; ' + cookie + '=')
  if (parts.length == 2) {
    return parts.pop().split(';').shift()
  } else {
    return false
  }
}


1			import $ from 'jquery'
2
3			export function getOptions (optionContainer) {
			0 ignored issues – show Unused Code introduced 2017-06-23 15:00 UTC by Report Bug Copy Issue Report The parameter `optionContainer` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
4			return {
5			expireDays: 7,
6			cookieName: 'cookies_accepted'
7			}
8			}
9
10			export function acceptCookies ($container, cookieName, expireDays) {
11			return function (e) {
			0 ignored issues – show Unused Code introduced 2017-06-23 15:00 UTC by Report Bug Copy Issue Report The parameter `e` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
12			const date = new Date()
13			date.setTime(date.getTime() + (expireDays2460601000))
14			document.cookie = cookieName + "=true; expires=" + date.toGMTString()
15
16			$container.remove()
17			}
18			}
19
20			export function checkCookies ($container, cookieName) {
21			const cookiesAccepted = getCookieValue(cookieName)
22
23			if (!cookiesAccepted) {
24			$container.addClass('cookieNotification-isVisible')
25			} else {
26			$container.remove()
27			}
28			}
29
30
31
32			function getCookieValue (cookie) {
33			const value = "; " + document.cookie
34			const parts = value.split('; ' + cookie + '=')
35			if (parts.length == 2) {
36			return parts.pop().split(';').shift()
37			} else {
38			return false
39			}
40			}
41

flyntwp / flynt-starter-theme

Push — BaseComponent-BlockCookieNotif... ( e5b0b7...17adf5 )

script.js ➔ acceptCookies A

Complexity

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like